Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Promote Pod Disruption Budgets to GA #27053

Merged
merged 1 commit into from
Mar 30, 2021

Conversation

mortent
Copy link
Member

@mortent mortent commented Mar 14, 2021

@k8s-ci-robot k8s-ci-robot added this to the 1.21 milestone Mar 14, 2021
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Mar 14, 2021
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Mar 14, 2021

Deploy preview for kubernetes-io-vnext-staging processing.

Building with commit d341b96

https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/60629404e39e8500089e4089

@k8s-ci-robot k8s-ci-robot added the language/en Issues or PRs related to English language label Mar 14, 2021
@k8s-ci-robot k8s-ci-robot added language/es Issues or PRs related to Spanish language language/ja Issues or PRs related to Japanese language language/ko Issues or PRs related to Korean language language/zh Issues or PRs related to Chinese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 14, 2021
@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 14, 2021
@mortent mortent marked this pull request as ready for review March 14, 2021 22:16
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 14, 2021
@reylejano
Copy link
Member

/assign
/sig apps

@k8s-ci-robot k8s-ci-robot added the sig/apps Categorizes an issue or PR as relevant to SIG Apps. label Mar 15, 2021
@reylejano
Copy link
Member

/remove-language es ja ko zh

@k8s-ci-robot k8s-ci-robot removed language/es Issues or PRs related to Spanish language language/ja Issues or PRs related to Japanese language language/ko Issues or PRs related to Korean language language/zh Issues or PRs related to Chinese language labels Mar 15, 2021
sftim
sftim previously requested changes Mar 16, 2021
Copy link
Contributor

@sftim sftim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mortent. Would you be willing to just change the English localization (en) here? SIG Docs localization teams pick up these changes independently from one another and each team can pick its own different process for managing their backlog.
We'll need this PR to omit changes to other localizations before we can merge it.
/hold

You might also want to check about including the changes for issue #26385 in this update. It's also OK if those come in as 2 PRs.

Optional extra tidying

  • add min-kubernetes-server-version: 1.21 to the front matter of content/en/docs/tasks/run-application/configure-pdb.md
    and add {{< version-check >}} into the prerequisites section
    • this is because older clusters don't have a PDB API available as policy/v1
  • remove “(available in Kubernetes 1.7 and higher)” as v1.7 is a long time out of support

content/en/docs/concepts/workloads/pods/disruptions.md Outdated Show resolved Hide resolved
content/en/docs/tasks/run-application/configure-pdb.md Outdated Show resolved Hide resolved
content/en/docs/tasks/run-application/configure-pdb.md Outdated Show resolved Hide resolved
@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 16, 2021
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 18, 2021
@mortent
Copy link
Member Author

mortent commented Mar 18, 2021

@sftim I have removed the changes to any other languages than English and updated the PR based on your suggestions.

@reylejano
Copy link
Member

Hi @mortent , thank you for having your Doc PR ready for review, here are upcoming doc related dates for the 1.21 release:
Upcoming doc related dates for the 1.21 release:

  • Docs Ready for Review deadline is March 24
  • Docs Ready to Merge deadline is March 31

@reylejano
Copy link
Member

@kubernetes/sig-apps-pr-reviews , please provide a technical review for this PR by March 31 to get this into the release. Thank you!

PodDisruptionBudgets. For policy/v1beta1 an empty selector matches zero pods, while
for policy/v1 an empty selector matches every pod in the namespace.
{{< /note >}}

{{< note >}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider omitting the “For versions 1.8 and earlier” note as v1.8 is long out of support.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the section.

@sftim sftim dismissed their stale review March 23, 2021 22:32

Earlier feedback was addressed

@@ -17,6 +18,8 @@ nodes.

## {{% heading "prerequisites" %}}

{{< version-check >}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recommend (not essential, but a good idea) that we ask people using older Kubernetes versions to check the documentation for that Kubernetes version.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I understand what we are looking for here. Is there an example I can look at?

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 30, 2021
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 30, 2021
@kow3ns
Copy link
Member

kow3ns commented Mar 30, 2021

/approve

@kow3ns
Copy link
Member

kow3ns commented Mar 30, 2021

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 30, 2021
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 0c55f99f6adf201fd077b9f4924d227d67e181f9

@tengqm
Copy link
Contributor

tengqm commented Mar 30, 2021

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kow3ns, tengqm

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 30, 2021
@reylejano
Copy link
Member

/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 30, 2021
@k8s-ci-robot k8s-ci-robot merged commit 0d2a4c7 into kubernetes:dev-1.21 Mar 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/apps Categorizes an issue or PR as relevant to SIG Apps. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants